Skip to content

refactor: add resource state icons and expose readiness#1330

Merged
edgehog-app[bot] merged 1 commit into
edgehog-device-manager:mainfrom
OmarBrbutovic:deployment_resource_state_icon
Apr 7, 2026
Merged

refactor: add resource state icons and expose readiness#1330
edgehog-app[bot] merged 1 commit into
edgehog-device-manager:mainfrom
OmarBrbutovic:deployment_resource_state_icon

Conversation

@OmarBrbutovic
Copy link
Copy Markdown
Collaborator

@OmarBrbutovic OmarBrbutovic commented Apr 3, 2026 •
edited
Loading

What this PR does / why we need it:

  • expose the readiness of resources in the API
  • add a new icon to represent the state of resources in the UI

Additional documentation e.g. usage docs, diagrams, reviewer notes, etc.:

Screenshots

Old:
image

Updated:
image
image
image

Thanks for sending a pull request! If this is your first time, here are some tips for you:
  1. You can take a look at our developer guide for an introduction on Edgehog development!
  2. Make sure to read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  3. If the PR is unfinished or you're actively working on it, mark it as draft

When fixing existing issues, use github's syntax to link your pull request to it

fixes #<issue number>

We also have a syntax to signal dependencies to other open pull requests

depends on #<pr number>
depends on https://github.com/...

In case of stacked PRs, you may add the PR number in the last commit's title instead:

gitGraph
    commit id: "Current master"
    branch feat1
    checkout feat1
    commit id: "feat: add something"
    commit id: "feat: add something else (#100)"
    branch feat2
    checkout feat2
    commit id: "refactor: do something"
    commit id: "fix: solve issue"
    commit id: "feat: add a feature (#101)"
    branch feat3
    checkout feat3
    commit id: "feat: feat without pr number"
Loading

@OmarBrbutovic OmarBrbutovic added the frontend This issue is relative to the Edgehog Frontend label Apr 3, 2026
@OmarBrbutovic OmarBrbutovic force-pushed the deployment_resource_state_icon branch 4 times, most recently from 93e4745 to 3c5574b Compare April 3, 2026 13:46
@ArnelaL ArnelaL mentioned this pull request Apr 7, 2026
Merged
@OmarBrbutovic OmarBrbutovic force-pushed the deployment_resource_state_icon branch from 3c5574b to 0ef78c7 Compare April 7, 2026 12:59
@OmarBrbutovic
refactor: add resource state icons and expose readiness
18be141 
- expose the readiness of resources in the API
- add a new icon to represent the state of resources in the UI

Signed-off-by: Omar <omar.brbutovic@secomind.com>
@OmarBrbutovic OmarBrbutovic force-pushed the deployment_resource_state_icon branch from 0ef78c7 to 18be141 Compare April 7, 2026 13:42
@ArnelaL
Copy link
Copy Markdown
Collaborator

ArnelaL commented Apr 7, 2026

/fast-forward

@edgehog-app edgehog-app Bot merged commit 18be141 into edgehog-device-manager:main Apr 7, 2026
35 checks passed
@OmarBrbutovic OmarBrbutovic deleted the deployment_resource_state_icon branch April 7, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ArnelaL ArnelaL ArnelaL approved these changes

+1 more reviewer

@JasminaPiric JasminaPiric JasminaPiric approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

frontend This issue is relative to the Edgehog Frontend

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@OmarBrbutovic @ArnelaL @JasminaPiric